Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

New template for RavenDB #367

Merged
merged 4 commits into from
Oct 3, 2024
Merged

New template for RavenDB #367

merged 4 commits into from
Oct 3, 2024

Conversation

hiddenshadow21
Copy link
Contributor

Template for monitoring RavenDB server using SNMP.

@igorbach
Copy link
Contributor

Hello!
Please, change the author name from RavenDB to yourself for easy maintenance in future

@gregolsky
Copy link

@igorbach We updated the authors. Anything else we need to do?

@igorbach
Copy link
Contributor

igorbach commented Sep 2, 2024

@gregolsky, there is a content error. You've provided the template_ravendb_server.yaml file, but it's content is JSON.
It is confusing and leads to import error.
Yaml is unified format for all Zabbix modern templates in this repo as well
Please, provide the correct template file

@gregolsky
Copy link

@igorbach Thank you for letting us know. We fixed the file. Is there anything else remaining please?

@gregolsky
Copy link

Please approve the workflow run.

@gregolsky
Copy link

can we merge this please?

@igorbach igorbach merged commit 47f59f9 into zabbix:main Oct 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants